Skip to content

[federation] Fix InstallPlan approval using ambiguous short name#3970

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:fix/federation-installplan-shortname-ambiguity
Jun 10, 2026
Merged

[federation] Fix InstallPlan approval using ambiguous short name#3970
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
lmiccini:fix/federation-installplan-shortname-ambiguity

Conversation

@lmiccini

@lmiccini lmiccini commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

The "oc get ip" command resolves to Kubernetes services instead of installplans.operators.coreos.com, causing the approval task to fail with "installplans.operators.coreos.com 172.30.0.1 not found" (a cluster service IP mistaken for an InstallPlan name).

Replace the shell command with Ansible-native tasks: extract the InstallPlan name from the already-registered ip_list variable using the same label filter as the wait task, then approve it via kubernetes.core.k8s.

The "oc get ip" command resolves to Kubernetes services instead of
installplans.operators.coreos.com, causing the approval task to fail
with "installplans.operators.coreos.com 172.30.0.1 not found" (a
cluster service IP mistaken for an InstallPlan name).

Replace the shell command with Ansible-native tasks: extract the
InstallPlan name from the already-registered ip_list variable using
the same label filter as the wait task, then approve it via
kubernetes.core.k8s.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Luca Miccini <lmiccini@redhat.com>
@lmiccini

lmiccini commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

tested downstream:

oc get installplan                                                                                                                                                                                                                                                                                   controller-0: Mon Jun  1 05:37:23 2026

NAME            CSV                             APPROVAL   APPROVED
install-m9bx5   rhsso-operator.7.6.12-opr-009   Manual     true

@evallesp evallesp left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@michburk michburk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@openshift-ci

openshift-ci Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michburk

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@lmiccini lmiccini removed request for danpawlik and vakwetu June 10, 2026 12:53
@openshift-merge-bot openshift-merge-bot Bot merged commit 9490655 into openstack-k8s-operators:main Jun 10, 2026
9 checks passed
@lmiccini lmiccini deleted the fix/federation-installplan-shortname-ambiguity branch June 10, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants